Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Specs for baked out functionality (redux) #51

Merged
merged 20 commits into from Dec 27, 2012
Merged

Specs for baked out functionality (redux) #51

merged 20 commits into from Dec 27, 2012

Conversation

christiannelson
Copy link
Contributor

No description provided.

christiannelson and others added 20 commits December 26, 2012 17:44
@christiannelson
Let's not include a password confirmation field.
3395c74
@christiannelson
Registration spec.
9c389d6
@christiannelson
Use the new expect() syntax. In fact, disallow the old syntax.
a3d4278
@christiannelson
No empty specs needed.
d87d94f
@christiannelson
Launchy is nice to have when debugging acceptance specs.
262aad9
@christiannelson
Cover registration happy path and rename feature spec files.
18ac4a3
@christiannelson
Sign out helper.
c14d8c8
@christiannelson
User mailcacher in development if it's running.
2d83fd6
@christiannelson
User emails are now unique
f95fc2a
@christiannelson
Comment tweaks
ff2b849
@squanto @christiannelson
Add activation acceptance coverage
bb2b2a5
@squanto @christiannelson
Add unsuccessful registration acceptance coverage
418c20b
@squanto @christiannelson
Fix authorization for password reset controller.
7737e61
@christiannelson
Add password reset coverage.
af1d333
@christiannelson
Consistent titlecasing on action buttons.
b0f959f
@christiannelson
Finishing touches on specs:
294f4ba 
- Better spec descriptions
- Use path/url helpers in feature specs
- Rename routes
@christiannelson
Generate controller specs using the new expect().to syntax.
80a3ee8
@christiannelson
Don't generate request specs by default.
42e2640
@christiannelson
Keep the default factory_girl info comment.
da13bde
@christiannelson
Update the change log.
f04ff3d
christiannelson added a commit that referenced this pull request Dec 27, 2012
@christiannelson
Merge pull request #51 from carbonfive/features/improve_coverage
c62f473 
Specs for baked out functionality and other spec-related enhancements.
@christiannelson christiannelson merged commit c62f473 into master Dec 27, 2012
@christiannelson christiannelson deleted the features/improve_coverage branch December 27, 2012 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@christiannelson @squanto